Skip to content

PCL: add some programs#91

Merged
kostis merged 3 commits intokostis:masterfrom
ntua-el21661:new-pcl-programs
Mar 1, 2026
Merged

PCL: add some programs#91
kostis merged 3 commits intokostis:masterfrom
ntua-el21661:new-pcl-programs

Conversation

@ntua-el21661
Copy link
Contributor

Add gcd and digits sum programs with inputs and outputs files

@kostis
Copy link
Owner

kostis commented Feb 28, 2026

Thanks for your contribution. I would like you to do two changes for this PR to be merged.

  1. The test suite already contains GCD programs (e.g. gcd.alan and gcd.dana) and these files come with specific input and result files. Rather than creating new such files, I suggest you modify your gcd.pcl program to read the input and write its output in the same way as the gcd.alan one and add symbolic links to the gcd.input and gcd.result files instead of having new ones.
  2. There is no digits_sum program in the test suite, so these input and result files should be new, but it's clear from your program that the result file will not be just a number but will also contain some message string. This needs to be added to the digits_sum.result file. (Note that it should have the extension .result, not .output.)

@kostis kostis merged commit d9f35ad into kostis:master Mar 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants